-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add stream_socket_test stream_write_test #231
base: master
Are you sure you want to change the base?
Conversation
EXPECT_EQ(sw->write(data.c_str(), 0), tateyama::status::ok); | ||
EXPECT_EQ(sw->write(nullptr, 0), tateyama::status::ok); | ||
EXPECT_EQ(sw->write(nullptr, INT_MIN), tateyama::status::ok); | ||
EXPECT_EQ(sw->write(nullptr, INT_MAX), tateyama::status::ok); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
このあたりのテストでstatus::okが戻るのは自明なので、できればcommitをするまでバッファに書かれない、commitでバッファされていたものが書かれる、というところが確認できるといいですね。ただ、既存のコードに変更が必要になるかもしれないので最終的にどこまでテストするかは @t-horikawa さんと相談してみてください。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
buffer_がprivate member variableなので
- buffer_ へのアクセス関数を用意
- フレンドクラスを使用する
の2択ですが、これはテストコード以外を修正する必要があるのでいったんこのテストコードをmergeして別のcommitとしてよいですか?
@kuron99 @t-horikawa アクセス関数とフレンドクラスの場合どちらがtsurugiの流儀になりますか?指定がなければsetterを用意してbuffer_を編集して、同時にgetterを用意してbuffer_の中身の確認するテストも追加します。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tsurugiの流儀と言えるものはない気がするのですが、私がやるとすればテスト用のアクセス関数を追加するほうですね。カプセル化を壊すのですが、コメントでテスト専用と書いて、本番コードからは呼ばないように気をつけると。
あとテストのやり方によってはgetterだけあればOKかもしれません。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@t-horikawa さんのapprovalもらったらマージしてOKです。
すいません。何をテストしようとしているのか良く分からないです。 なので、マージはpendingとさせてください。改良案は私も考えます。 |
Test for 365d48e (Improve TCP connection by buffering data for faster writes)